Fix exportToSQLite errors caused by old table name and parameter order #159
Conversation
|
@mccullen Thanks so much for submitting this PR! The parameter order makes sense. The table name issues are a combination of typos and this function originally supporting version only OMOP v5.3. The list of tables you provided support OMOP v5.4 which is great but we'd want to add some logic (like we do in other functions here when defining the event_tables variable to support both versions of the OMOP data model. Would you be able to edit this in the PR? Alternatively, we can add this as an update on our side. Thanks again for your input! |
Oh, sorry for the late reply. Sure, I will have a look when I get the chance and make those updates. |
I was having issues with the exportToSQLite function and had to make these changes to get it to work. I assume the issue was that this function is using an old CDM table name (cohort_attribute?) and some parameter orders have changed. Either that or I'm not doing something right on my end.